Skip to content

Fix UI/Layout Inconsistencies Across Project Cards and Modal Rendering (#595)#691

Merged
steam-bell-92 merged 3 commits into
steam-bell-92:mainfrom
Mudita-Singh:issue/ui-modal-card-layout-fix
May 23, 2026
Merged

Fix UI/Layout Inconsistencies Across Project Cards and Modal Rendering (#595)#691
steam-bell-92 merged 3 commits into
steam-bell-92:mainfrom
Mudita-Singh:issue/ui-modal-card-layout-fix

Conversation

@Mudita-Singh
Copy link
Copy Markdown
Contributor

Related Issue

Closes #595

Description

This PR addresses multiple UI and layout inconsistencies found across project cards and modal rendering.

Changes Made

  • Standardized modal sizing and content scaling across projects.
  • Fixed excessive whitespace appearing in certain modal layouts.
  • Improved card alignment and spacing consistency.
  • Unified responsive behavior across different screen sizes.
  • Removed conflicting styling causing visual inconsistencies.
  • Improved theme compatibility and consistency across components.

Testing Performed

  • Verified rendering across multiple project cards.
  • Tested modal layouts on different screen sizes.
  • Checked dark/light theme behavior.
  • Confirmed consistent spacing, alignment, and scaling.
  • Ensured no regressions in existing functionality.

Before

  • Inconsistent modal scaling between projects.
  • Uneven spacing and alignment.
  • Excess empty space in some modals.
  • Responsive layouts behaved differently across sections.

After

  • Consistent modal dimensions and rendering.
  • Uniform card spacing and alignment.
  • Improved responsiveness.
  • Consistent visual experience across all projects.

Checklist

@vercel
Copy link
Copy Markdown

vercel Bot commented May 23, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
python-mini-project Ready Ready Preview, Comment May 23, 2026 5:41pm

@steam-bell-92
Copy link
Copy Markdown
Owner

@Mudita-Singh great work with those adjustments but kind of our current images instead of logos are, lexiographic arrangement of cards are gone. We need those changes

@Mudita-Singh
Copy link
Copy Markdown
Contributor Author

Thanks for reviewing. I'll update the PR to preserve the existing project images/logos and restore the lexicographic card arrangement while keeping the UI/layout improvements. I'll push the changes shortly.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 23, 2026

@Mudita-Singh is attempting to deploy a commit to the Anuj's projects Team on Vercel.

A member of the Team first needs to authorize it.

…-layout-fix

# Conflicts:
#	web-app/index.html
@Mudita-Singh
Copy link
Copy Markdown
Contributor Author

@steam-bell-92 I've pushed an update addressing the issues mentioned in the review. The project card media rendering has been restored, and the project cards are now back in their intended lexicographic (alphabetical) order. The UI/layout consistency improvements remain unchanged.

@steam-bell-92 steam-bell-92 merged commit 602e402 into steam-bell-92:main May 23, 2026
7 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

🎉 Thank you for your contribution!

Your Pull Request has been merged successfully.

We appreciate the time and effort you put into improving this project. Contributions like yours help the repository grow and stay useful for everyone.

If you'd like to contribute again, please check the open issues and make sure you are assigned before opening another Pull Request.

Thanks again for your support! 🙌

@steam-bell-92
Copy link
Copy Markdown
Owner

@Mudita-Singh Great work!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

🐛 [Bug]: Multiple UI/layout inconsistencies across project cards and modal rendering

2 participants